Skip to content

Conversation

@mario-campos
Copy link

No description provided.

@mario-campos mario-campos requested a review from Copilot October 23, 2025 03:31
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds a change note documenting the addition of models and tests for SQLite libraries (sqlite and sqlite_async). The change note follows the standard format for test-related updates.

@mario-campos mario-campos requested a review from Copilot October 23, 2025 03:33
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comment on lines +4 to +5
* Added models for the `sqlite` and `sqlite_async` libraries.
* Added tests for the new SQLite models as well. No newline at end of file
Copy link

Copilot AI Oct 23, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[nitpick] The second bullet point is redundant. Consider combining both points into a single concise statement, such as: 'Added models and tests for the sqlite and sqlite_async libraries.'

Suggested change
* Added models for the `sqlite` and `sqlite_async` libraries.
* Added tests for the new SQLite models as well.
* Added models and tests for the `sqlite` and `sqlite_async` libraries.

Copilot uses AI. Check for mistakes.
@mario-campos
Copy link
Author

@rachmari claims bug is fixed. Re-testing...

@mario-campos mario-campos requested a review from Copilot October 24, 2025 19:08
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

@@ -0,0 +1,5 @@
---
category: test
Copy link

Copilot AI Oct 24, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The category 'test' is not a valid category for library-pack change notes. Based on the description ('Added models for the sqlite and sqlite_async libraries'), this should be categorized as 'feature' since it adds new library APIs/models.

Copilot generated this review using guidance from repository custom instructions.
@mario-campos
Copy link
Author

Changed change-note.instructions.md to refer to docs/change-notes.md.

@mario-campos mario-campos requested a review from Copilot October 24, 2025 19:17
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants